Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding SocketIO handshake data to service call parameters #49

Merged
merged 1 commit into from
Apr 3, 2014

Conversation

daffl
Copy link
Member

@daffl daffl commented Apr 3, 2014

This is related to #44, #48 and #32 adding SocketIO handshake data as the service parameters like:

app.configure(feathers.socketio(function(io) {
  io.set('authorization', function (handshake, callback) {
    handshake.feathers.user = { name: 'David' };
  });
}));

With something like:

app.use('todos', {
  create: function(data, params, callback) {
    // When called via SocketIO:
    params.user // -> { name: 'David' }
  }
});

@daffl daffl added this to the 0.4.0 milestone Apr 3, 2014
@daffl daffl changed the title Adding SocketIO handshake data to service call parameters. Adding SocketIO handshake data to service call parameters Apr 3, 2014
daffl added a commit that referenced this pull request Apr 3, 2014
Adding SocketIO handshake data to service call parameters
@daffl daffl merged commit 04b3931 into master Apr 3, 2014
@daffl daffl deleted the socket-params branch April 3, 2014 23:26
daffl pushed a commit that referenced this pull request Aug 19, 2018
* chore(package): update dependencies

https://greenkeeper.io/

* docs(readme): add Greenkeeper badge

https://greenkeeper.io/

* Update to latest Semistandard
daffl pushed a commit that referenced this pull request Aug 21, 2018
* chore(package): update dependencies

https://greenkeeper.io/

* docs(readme): add Greenkeeper badge

https://greenkeeper.io/

* Update to latest Semistandard
daffl pushed a commit that referenced this pull request Aug 25, 2018
daffl added a commit that referenced this pull request Aug 25, 2018
* Update plugin generator to correspond with latest release

* Add mocha to Semistandard env
daffl added a commit that referenced this pull request Aug 29, 2018
daffl pushed a commit that referenced this pull request Aug 29, 2018
daffl added a commit that referenced this pull request Aug 29, 2018
daffl pushed a commit that referenced this pull request Aug 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant